Skip to content

mmap_registers: Made mmap functionality conditional on mmap support#3294

Closed
bgoing-micron-oss wants to merge 1 commit intolinux-nvme:masterfrom
Micron-TPG-OSS:bgoing/conditional-mmap-support
Closed

mmap_registers: Made mmap functionality conditional on mmap support#3294
bgoing-micron-oss wants to merge 1 commit intolinux-nvme:masterfrom
Micron-TPG-OSS:bgoing/conditional-mmap-support

Conversation

@bgoing-micron-oss
Copy link
Copy Markdown
Contributor

Bypassed the mmap functionality in mmap_registers if the target platform doesn't support mmap. Created munmap_registers for conditional unmap usage as well.

Used NVME_HAVE_MMAP to avoid conflict with python's HAVE_MMAP definition when building python wrappers.

Bypassed the mmap functionality in mmap_registers if the target platform
doesn't support mmap.  Created munmap_registers for conditional unmap
usage as well.

Used NVME_HAVE_MMAP to avoid conflict with python's HAVE_MMAP definition
when building python wrappers.

Signed-off-by: Broc Going <[email protected]>
@igaw
Copy link
Copy Markdown
Collaborator

igaw commented Apr 21, 2026

I've updated the commit message so it uses present tense and applied it directly. Thanks!

@igaw igaw closed this Apr 21, 2026
@bgoing-micron-oss bgoing-micron-oss deleted the bgoing/conditional-mmap-support branch April 21, 2026 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants